feat: Add drag-and-drop functionality to snippets table #318
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
Just noting a styling change that seems to be caused by this PR.
|
Contributor
Author
|
Okay I'll have a look. |
Contributor
Author
|
Are the tests bugged? Even on core branch, 7 of them pass. I can't really replicate the bug you attached, or pass all the tests. The styling bug happened while you ran the tests, or you tested it manually? |
Contributor
|
@mgiannopoulos24 Sorry it seems this was not related to your changes! |
louiswol94
approved these changes
Dec 18, 2025
Contributor
Author
|
@louiswol94 Have you developed new tests because these fail? I've seen on some other PR that you pass them all. |
Contributor
|
@mgiannopoulos24 Merged. Thank you so much for the contribution! 🚀 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description:
This PR adds the ability to reorder snippets via drag-and-drop on the "All Snippets" administration page. This improves the UX for managing snippet execution order, replacing the need to manually edit priority numbers.
Related issue:
Issue #314
Changes
src/js/services/manage/ordering.tsto initialize jQuery UI Sortable on the snippets table. It calculates the new priority based on the row index and sends an AJAX request to update the database immediately upon drop.src/css/manage.scssto change the cursor to move on hover and style the sortable placeholder (dashed border) for better visual feedback.src/js/manage.tsto initialize the ordering service.